Replace some assert_throws("Something", stuff) calls with assert_throws_dom. (#21377) This diff was generated by running: find . -type f -print0 | xargs -0 perl -pi -e 'BEGIN { $/ = undef; } s/assert_throws\(([ \n]*"[A-Za-z_]*") *(, *.)/assert_throws_dom(\1\2/gs' followed by a fix to xhr/resources/send-after-setting-document-domain-window-helper.js to make xhr/send-after-setting-document-domain.htm pass. This does affect indentation poorly in cases when the first arg was on the same line as the assert_throws, there was a newline after the ',' after the first arg, and the following args were lined up with the first arg. Fixing that, especially when there are multiple lines after the first arg, is not trivial with a regexp. Co-authored-by: Boris Zbarsky <bzbarsky@mit.edu> diff --git a/user-timing/measure-exceptions.html b/user-timing/measure-exceptions.html index 0c532e7..2836eae 100644 --- a/user-timing/measure-exceptions.html +++ b/user-timing/measure-exceptions.html
@@ -27,7 +27,7 @@ ]; eventMarks.forEach(function(name) { test(()=>{ - assert_throws("InvalidAccessError", ()=>{ + assert_throws_dom("InvalidAccessError", ()=>{ window.performance.measure("measuring", name, "mark"); }, "Should throw"); }, `Passing '${name}' as a mark to measure API should cause error when the mark is empty.`); @@ -39,7 +39,7 @@ ]; args.forEach(each => { test(()=>{ - assert_throws("SyntaxError", ()=>{ + assert_throws_dom("SyntaxError", ()=>{ window.performance.measure("measuring", each, "mark"); }, "Should throw"); }, `Passing ${each} as a mark to measure API should cause error.`); 